-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue #11665: Create a robust method of surviving mutation suppression #11731
Conversation
822364a
to
94c63c1
Compare
@Vyom-Yadav let's add codenarc to CI execution now that we are adding groovy to main repo. Take a look at contribution to see how it is done there: https://github.com/checkstyle/contribution/blob/master/checkstyle-tester/codenarc.sh Issue created at #11734 |
0fae0d0
to
1b22465
Compare
8751d77
to
82b51fc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some early feedback:
0e36130
to
e7e3df3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Items
c7de5b5
to
73f7fee
Compare
81a3fa3
to
fba897f
Compare
8b79115
to
9ea8a0d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok to merge
9ea8a0d
to
920205f
Compare
920205f
to
0ab0169
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can deal with team city failures later, let's get this merged and kill some mutants.
Resolves #11665
Only the last commit is under review.
Proofs ->
Here is the difference between old pitest suppressions and new pitest suppressions.
The topmost mutation
}
is due to hcoles/pitest#688 (See this) and the 6 added mutations were multiple mutations on the same line. Report is good according to me. A mix ofsed
,awk
,groovy
was used, report is accurate.